-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
V1.2.1 #35
Conversation
closes #33 |
Codecov Report
@@ Coverage Diff @@
## master #35 +/- ##
===========================================
Coverage 100.00% 100.00%
Complexity 3 3
===========================================
Files 1 1
Lines 7 7
===========================================
Hits 7 7 Continue to review full report at Codecov.
|
Dockerfile
Outdated
USER las2peer | ||
RUN chmod -R a+rwx /src | ||
RUN chmod +x /src/docker-entrypoint.sh | ||
RUN chmod +x gradlew && ./gradlew build --exclude-task test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is the test task excluded?
docker-entrypoint.sh
Outdated
function set_in_service_config { | ||
sed -i "s?${1}[[:blank:]]*=.*?${1}=${2}?g" ${SERVICE_PROPERTY_FILE} | ||
} | ||
#cp $SERVICE_PROPERTY_FILE $SERVICE_PROPERTY_FILE |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is this line for?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
which line do you mean?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The line that is commented out
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dunno, removed it
docker-entrypoint.sh
Outdated
function set_in_service_config { | ||
sed -i "s?${1}[[:blank:]]*=.*?${1}=${2}?g" ${SERVICE_PROPERTY_FILE} | ||
} | ||
#cp $SERVICE_PROPERTY_FILE $SERVICE_PROPERTY_FILE |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The line that is commented out
Dockerfile
Outdated
EXPOSE $HTTPS_PORT | ||
EXPOSE $LAS2PEER_PORT | ||
RUN chmod +x /src/docker-entrypoint.sh | ||
RUN chmod +x docker-entrypoint.sh |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Either this line or the one above is not needed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed it now
Dockerfile
Outdated
RUN chmod +x gradlew && ./gradlew build | ||
#RUN dos2unix /src/docker-entrypoint.sh | ||
#RUN dos2unix /src/etc/i5.las2peer.connectors.webConnector.WebConnector.properties | ||
#RUN dos2unix /src/etc/i5.las2peer.services.servicePackage.akgService.properties |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe add a comment here telling users why this is commented out
Added template dockerfile
Updated core to version 1.2.1